Skip to content

fix(files): correctly sort views#58330

Merged
susnux merged 2 commits intomasterfrom
fix/view-order
Feb 14, 2026
Merged

fix(files): correctly sort views#58330
susnux merged 2 commits intomasterfrom
fix/view-order

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Feb 13, 2026

Summary

The condition to sort by order was missing, so the views were only sorted by name. Added the most important order condition and proper tests for this.

Checklist

@susnux susnux added this to the Nextcloud 34 milestone Feb 13, 2026
@susnux susnux requested review from a team and skjnldsv as code owners February 13, 2026 14:44
@susnux susnux added bug 3. to review Waiting for reviews labels Feb 13, 2026
@susnux susnux removed the request for review from a team February 13, 2026 14:44
@susnux susnux requested review from nfebe and sorbaugh February 13, 2026 14:44
@susnux
Copy link
Contributor Author

susnux commented Feb 13, 2026

/backport to stable33

@susnux susnux requested review from artonge and cristianscheid and removed request for sorbaugh February 13, 2026 14:44
The condition to sort by order was missing, so the views were only
sorted by name. Added the most important order condition and proper
tests for this.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
Copy link
Contributor Author

susnux commented Feb 14, 2026

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@susnux susnux merged commit 66c1e0a into master Feb 14, 2026
126 checks passed
@susnux susnux deleted the fix/view-order branch February 14, 2026 21:59
@susnux
Copy link
Contributor Author

susnux commented Feb 14, 2026

/backport 2007491 to stable33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants